Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update nexus link converter #6460

Merged
merged 1 commit into from
Aug 28, 2024
Merged

Conversation

rodrigozhou
Copy link
Contributor

What changed?

Update nexus link converter

Why?

Sync with code in SDK.

How did you test it?

Potential risks

Documentation

Is hotfix candidate?

@rodrigozhou rodrigozhou requested a review from bergundy August 28, 2024 17:26
@rodrigozhou rodrigozhou requested a review from a team as a code owner August 28, 2024 17:26
@rodrigozhou rodrigozhou force-pushed the rodrigozhou/fix-nexus-link-converter branch from f945868 to e1499aa Compare August 28, 2024 18:23
@rodrigozhou rodrigozhou force-pushed the rodrigozhou/fix-nexus-link-converter branch from e1499aa to 2f42c84 Compare August 28, 2024 20:28
@rodrigozhou rodrigozhou merged commit 20e519f into main Aug 28, 2024
41 of 42 checks passed
@rodrigozhou rodrigozhou deleted the rodrigozhou/fix-nexus-link-converter branch August 28, 2024 21:41
lina-temporal pushed a commit that referenced this pull request Aug 28, 2024
## What changed?
<!-- Describe what has changed in this PR -->
Update nexus link converter

## Why?
<!-- Tell your future self why have you made these changes -->
Sync with code in SDK.

## How did you test it?
<!-- How have you verified this change? Tested locally? Added a unit
test? Checked in staging env? -->

## Potential risks
<!-- Assuming the worst case, what can be broken when deploying this
change to production? -->

## Documentation
<!-- Have you made sure this change doesn't falsify anything currently
stated in `docs/`? If significant
new behavior is added, have you described that in `docs/`? -->

## Is hotfix candidate?
<!-- Is this PR a hotfix candidate or does it require a notification to
be sent to the broader community? (Yes/No) -->
dnr pushed a commit that referenced this pull request Oct 4, 2024
## What changed?
<!-- Describe what has changed in this PR -->
Update nexus link converter

## Why?
<!-- Tell your future self why have you made these changes -->
Sync with code in SDK.

## How did you test it?
<!-- How have you verified this change? Tested locally? Added a unit
test? Checked in staging env? -->

## Potential risks
<!-- Assuming the worst case, what can be broken when deploying this
change to production? -->

## Documentation
<!-- Have you made sure this change doesn't falsify anything currently
stated in `docs/`? If significant
new behavior is added, have you described that in `docs/`? -->

## Is hotfix candidate?
<!-- Is this PR a hotfix candidate or does it require a notification to
be sent to the broader community? (Yes/No) -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants